[ResourceTiming] Cleanup buffer-full tests Change-Id: Ic5f9968f5c0a1c9b5ad4760f4aa2d8ee85c73c40 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2911104 Commit-Queue: Nicolás Peña Moreno <npm@chromium.org> Reviewed-by: Tom McKee <tommckee@chromium.org> Reviewed-by: Yoav Weiss <yoavweiss@chromium.org> Cr-Commit-Position: refs/heads/master@{#887367} diff --git a/resource-timing/buffer-full-decrease-buffer-during-callback.html b/resource-timing/buffer-full-decrease-buffer-during-callback.html index e6de33d..3091fcf 100644 --- a/resource-timing/buffer-full-decrease-buffer-during-callback.html +++ b/resource-timing/buffer-full-decrease-buffer-during-callback.html
@@ -3,46 +3,23 @@ <head onload> <meta charset="utf-8" /> <title>This test validates that decreasing the buffer size in onresourcetimingbufferfull callback does not result in extra entries being dropped.</title> -<link rel="help" href="http://www.w3.org/TR/resource-timing/#performanceresourcetiming"/> +<link rel="author" title="Google" href="http://www.google.com/" /> +<link rel="help" href="https://www.w3.org/TR/resource-timing-2/#dom-performance-onresourcetimingbufferfull"/> <script src="/resources/testharness.js"></script> <script src="/resources/testharnessreport.js"></script> +<script src="resources/resource-loaders.js"></script> <script src="resources/buffer-full-utilities.js"></script> </head> <body> <script> -const resource_timing_buffer_size = 2; -let eventFired = false; -setup(() => { - // Get the browser into a consistent state. - clearBufferAndSetSize(resource_timing_buffer_size); - let resize = () => { - performance.setResourceTimingBufferSize(resource_timing_buffer_size - 1); - eventFired = true; - } - performance.addEventListener('resourcetimingbufferfull', resize); -}); - -let overflowTheBuffer = () => { - return new Promise(resolve => { - // This resource overflows the entry buffer, and goes into the secondary buffer. - // Since the buffer size doesn't increase, it will eventually be dropped. - appendScript('resources/empty_script.js', resolve); - }); -}; - -let testThatBufferContainsTheRightResources = () => { - let entries = performance.getEntriesByType('resource'); - assert_equals(entries.length, 2, - 'Both entries should be stored in resource timing buffer since it decreased its limit only after it overflowed.'); - assert_true(entries[0].name.includes('empty.js'), "empty.js is in the entries buffer"); - assert_true(entries[1].name.includes('empty.js?second'), "empty.js?second is in the entries buffer"); -}; - promise_test(async () => { - await fillUpTheBufferWithTwoResources('resources/empty.js'); - await overflowTheBuffer(); - await waitForEventToFire(); - testThatBufferContainsTheRightResources(); + performance.addEventListener('resourcetimingbufferfull', () => { + performance.setResourceTimingBufferSize(1); + }); + await fillUpTheBufferWithTwoResources(); + load.script(scriptResources[2]); + await bufferFullFirePromise; + checkEntries(2); }, "Test that decreasing the buffer limit during the callback does not drop entries"); </script> </body> 